Skip to content

Remove deprecated IServerContainer - #64095

Open
come-nc wants to merge 11 commits into
masterfrom
fix/cleanup-server-container
Open

Remove deprecated IServerContainer#64095
come-nc wants to merge 11 commits into
masterfrom
fix/cleanup-server-container

Conversation

@come-nc

@come-nc come-nc commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove deprecated IServerContainer interface and associated methods, cleanup resulting code a bit.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@come-nc come-nc self-assigned this Sep 7, 2026
@come-nc come-nc added the 2. developing Work in progress label Sep 7, 2026
@come-nc come-nc added this to the Nextcloud 36 milestone Sep 7, 2026
@CarlSchwan

Copy link
Copy Markdown
Member

Noice :)

@come-nc
come-nc force-pushed the fix/cleanup-server-container branch 2 times, most recently from ba8033c to 5e685dd Compare September 7, 2026 16:14
@come-nc come-nc added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 performance 🚀 and removed 2. developing Work in progress labels Sep 7, 2026
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…asses

Switch query to protected and remove it from public API. Optimize a bit.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…rvice

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…ervice

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Supposedly this is slightly better for performances as the closure is
 not bound to the object.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the fix/cleanup-server-container branch from 5e685dd to f2d705c Compare September 10, 2026 09:58
@come-nc
come-nc marked this pull request as ready for review September 10, 2026 12:46
@come-nc
come-nc requested a review from a team as a code owner September 10, 2026 12:46
@come-nc
come-nc requested review from Altahrim, icewind1991, leftybournes and provokateurin and removed request for a team September 10, 2026 12:46
@come-nc
come-nc requested a review from CarlSchwan September 10, 2026 14:33
@come-nc
come-nc added this pull request to stack #64192 September 10, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews performance 🚀 technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants